Skip to content

Enable syntax highlighting for Maven snippet. #36

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 1 commit into from
Closed

Enable syntax highlighting for Maven snippet. #36

wants to merge 1 commit into from

Conversation

johnjohndoe
Copy link

No description provided.

@patriknw
Copy link
Contributor

LGTM

@rkuhn
Copy link
Member

rkuhn commented May 5, 2014

Thanks! Waiting for #28 before we can merge this.

@rkuhn
Copy link
Member

rkuhn commented May 20, 2014

@johnjohndoe Now that we have the rules in place, please include your copyright waiver as described there.

To my mind we can merge such pure formatting changes that do not modify the meaning without a full vote, but in case others disagree I register my LGTM (condition on the copyright waiver being added).

@viktorklang
Copy link
Contributor

Vote to close due to no action in 2 months.

@johnjohndoe
Copy link
Author

Sorry, the rules are a bit too much for some syntax highlighting. Feel free to apply the change as an own commit.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants